-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
unittest for common/scene.py #371
Conversation
Add tianchi-competition-2024 label Now! |
提交代码后需要确认是否报名成功了,步骤如下: |
ok
…________________________________
发件人: ***@***.*** ***@***.***> 代表 jingshun.tq ***@***.***>
发送时间: 2024年8月1日 00:24
收件人: oceanbase/obdiag ***@***.***>
抄送: YSevenK ***@***.***>; Author ***@***.***>
主题: Re: [oceanbase/obdiag] unittest for common/scene.py (PR #371)
提交代码后需要确认是否报名成功了,步骤如下:
报名参赛步骤:报名——下载选手方案提交模版——提交结果
赛事报名官网:https://tianchi.aliyun.com/competition/entrance/532215/submission/1282<https://tianchi.aliyun.com/competition/entrance/532215/submission/1282%E2%80%8B>
image.png (view on web)<https://github.com/user-attachments/assets/64151e2c-5271-4ecc-ae8a-445242a8d38d>
表格内容要求:
1. 选手需提供GitHub个人设置页面截图和通义灵码辅助编码使用截图(此项在提交模板里有参考截图示意。)
2. 相关内容需要填写:
image.png (view on web)<https://github.com/user-attachments/assets/754f10d7-e613-480e-b4a2-a89172def27b>
将参赛表格内容填写后,提交到【提交结果】中
image.png (view on web)<https://github.com/user-attachments/assets/751163f2-5dcd-4256-bc3e-708e50270224>
如有疑问可加入赛事交流群:钉钉群号:75530009710
—
Reply to this email directly, view it on GitHub<#371 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AYKG7VA7GWWM7DATTFNQWEDZPHPC3AVCNFSM6AAAAABLQAYKICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGIZTOMZXGU>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
好,我后面看一下 |
代码格式化的流水线没跑过,没有格式化代码。需要执行一下 black --check -S -l 256 . 参考文档:https://oceanbase.yuque.com/org-wiki-obtech-vh7w9r/imzr6c/dio3s4re84gz54pp |
已完成 |
good job! |
test/__init__.py
Outdated
@time: 2022/6/20 | ||
@file: __init__.py | ||
@desc: | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个文件删了吧
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
刚才试了一下,删除这个文件会报错
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这边我帮你删了,后面的特性分支可以单独建一个分支去开发,尽量不要去在fork的仓库master分支上开发,因为master更适合在作为反合的目的分支进行处理,这样可以更好地获取源仓库里最新代码。
其他没什么问题了,祝获得好成绩~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好的,因为没有什么参与开源的经验,您提到的后面会注意,也祝您工作顺利 NO BUG!
Summary
为
common/scene.py
文件添加测试 到test/common/test_scene.py
目录下Solution Description
使用通义灵码的单元测试功能进行开发